WIP: Sync with titiler #7
Conversation
f-skold
force-pushed
the
sync_with_titiler
branch
from
4e59f0e to
7052c36
Compare
|
I am pretty new to this. It was a few weeks since I did look at this the previous time. (Now I checked the logs on AWS and saw that it complains like this:) Keept working on it and not it seems better |
f-skold
force-pushed
the
sync_with_titiler
branch
from
7052c36 to
5ed0df5
Compare
|
If I try to deploy 5ed0df5, I can deploy it, but when I do try to use it (get a tilejson for sentinel 2), I got exception when trying to parse the sceneid
And adding more debugging (catching the eexception in rio-tiler-pds), this give strong indications that we try to use a
This could be solved in several ways. Also, to me it seems like parsing of sceneid is done both in |
Change language_version from python3.7 to any python
Bump to black 23.11.0.
Remove extra comma in CustomPathParams.__post_init__() causing reformating.
Reformated all code using black:
git ls-files | grep "\.py" | xargs touch
pre-commit run -v --all-files
WIP: When running this. There is an issue that the CustomPathParams with two members
sceneid and scene_metadata is passed as path_dependency.
The user of that dependency assumes to get a string that is interpreted to be a sceneid, not get an object.
**I would need some gudience on what changes to make.**
Update version of (this) package to 0.1.0
Use rio-tiler-pds and titiler 0.7.0.
Advance aws-cdk version to 1.160.0
The titiler package has been split into three packages, do refer to each of them.
Update python imports and usage.
(When trying to build using the old Dockerfile, the output was too big to be used in the old way.)
Change to build a docker imaged based on AWS python image instead of creating a bundle that is
overlayed on the AWS python environment.
Changed several things in stack/app.py:
* One thing had changed name to apigw_integrations.HttpLambdaIntegration
and required a new id argument.
* Use a docker image instead of using docker image to construct a usable zip for the python runtime
FROM_IMAGE.
Add wheel to use with corrections Update dockerfile Try to see if we could pass the correct thing.
f-skold
force-pushed
the
sync_with_titiler
branch
from
5ed0df5 to
c6bc1d6
Compare
NOTE: When I do these updates, the result of the docker build get to big to be uploaded to AWS Lambda (which was my primary target)
So I would like help / suggestions on how to get something that can be used on AWS Lambda
What I am changing
The goal is to update to current versions of the depedencies.
How I did it
NOTE: Deploying to AWS Lambda fails since reult is to large to be used by lambda.
How you can test it
Check out, try to deploy to lambda, failure.
Should be no functional changes.
Related Issues
#6 might be helped by this since I advance some packages